Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Diffusers transformers version update #1029

Merged

Conversation

yeounoh
Copy link
Contributor

@yeounoh yeounoh commented Dec 19, 2023

Description

In the previous PR, #1027, I used 4.26.0 which should have been 4.36.0.

Tests

I ran one-shot to confirm: https://pantheon.corp.google.com/kubernetes/job/us-central2/xl-ml-test-us-central2/default/pt-nightly-hf-diffusers-conv-v4-8-1vm-cgpqv/logs?mods=allow_workbench_image_override&project=xl-ml-test, with a shorter number of steps -- just to confirm that the model saving at the end succeeds:

Steps:  95%|█████████▌| 19/20 [15:32<00:02,  2.64s/it, lr=1e-5, step_loss=0.00433]INFO:root:Joining rendezvous 'accelerate.utils.wait_for_everyone'...
INFO:root:Joining rendezvous 'accelerate.utils.wait_for_everyone'...
INFO:root:Joining rendezvous 'accelerate.utils.wait_for_everyone'...
Steps: 100%|██████████| 20/20 [15:34<00:00,  2.42s/it, lr=1e-5, step_loss=0.0322] INFO:root:Joining rendezvous 'accelerate.utils.wait_for_everyone'...
model_index.json: 100%|██████████| 541/541 [00:00<00:00, 2.85MB/s]
safety_checker/config.json: 100%|██████████| 4.56k/4.56k [00:00<00:00, 11.7MB/s]
(…)kpoints/scheduler_config-checkpoint.json: 100%|██████████| 209/209 [00:00<00:00, 1.71MB/s]
(…)ature_extractor/preprocessor_config.json: 100%|██████████| 342/342 [00:00<00:00, 1.97MB/s]
model.safetensors: 100%|██████████| 1.22G/1.22G [00:04<00:00, 247MB/s] [00:00<?, ?B/s]
Fetching 10 files: 100%|██████████| 10/10 [00:05<00:00,  1.81it/s]/342 [00:00<?, ?B/s]
{'requires_safety_checker', 'image_encoder'} was not found in config. Values will be initialized to default values.
                                                                     Loaded feature_extractor as CLIPImageProcessor from `feature_extractor` subfolder of CompVis/stable-diffusion-v1-4.
Loaded tokenizer as CLIPTokenizer from `tokenizer` subfolder of CompVis/stable-diffusion-v1-4.
{'prediction_type', 'timestep_spacing'} was not found in config. Values will be initialized to default values.
Loaded scheduler as PNDMScheduler from `scheduler` subfolder of CompVis/stable-diffusion-v1-4.
`text_config_dict` is provided which will be used to initialize `CLIPTextConfig`. The value `text_config["id2label"]` will be overriden.
`text_config_dict` is provided which will be used to initialize `CLIPTextConfig`. The value `text_config["bos_token_id"]` will be overriden.
`text_config_dict` is provided which will be used to initialize `CLIPTextConfig`. The value `text_config["eos_token_id"]` will be overriden.
Loaded safety_checker as StableDiffusionSafetyChecker from `safety_checker` subfolder of CompVis/stable-diffusion-v1-4.
Loading pipeline components...: 100%|██████████| 7/7 [00:00<00:00, 14.53it/s]
Steps: 100%|██████████| 20/20 [15:41<00:00, 47.09s/it, lr=1e-5, step_loss=0.0322]
+ exit_code=0
+ bash /scripts/cleanup.sh
Delete request issued for: [tpu-3d4a6e12-33bd-4ce1-b6d4-fd21718962f7]
Check operation [projects/xl-ml-test/locations/us-central2-b/operations/operation-1703031745290-60ce5fa38200a-a57bcc28-88e6367c] for status.

@yeounoh yeounoh self-assigned this Dec 19, 2023
@yeounoh yeounoh marked this pull request as draft December 19, 2023 21:51
@yeounoh yeounoh requested a review from RissyRan December 20, 2023 00:58
@yeounoh yeounoh marked this pull request as ready for review December 20, 2023 00:58
@yeounoh yeounoh requested a review from will-cromar December 20, 2023 01:03
@yeounoh yeounoh requested review from will-cromar and removed request for will-cromar December 20, 2023 01:08
@RissyRan RissyRan merged commit 1d66e87 into GoogleCloudPlatform:master Dec 20, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants